Skip to content

Conversation

@echoix
Copy link
Member

@echoix echoix commented Jun 22, 2025

  • Added translation comments for context on the units

  • Made strings reorderable for translation
    A few strings were flagged by when running xgettext, mentioning that format string with unnamed arguments cannot be properly localized:
    The translator cannot reorder the arguments.
    Please consider using a format string with named arguments,
    and a mapping instead of a tuple for the arguments.

  • Also shoved in fixes of some typos encountered during that search.

echoix added 4 commits June 22, 2025 03:58
A few strings were flagged by when running xgettext, mentioning that format string with unnamed arguments cannot be properly localized:
The translator cannot reorder the arguments.
Please consider using a format string with named arguments,
and a mapping instead of a tuple for the arguments.
@echoix echoix added the translation Message translation related label Jun 22, 2025
@github-actions github-actions bot added GUI wxGUI related vector Related to vector data processing raster Related to raster data processing Python Related code is in Python C Related code is in C database Related to database management libraries module display imagery labels Jun 22, 2025
@echoix echoix requested a review from nilason June 23, 2025 23:05
Copy link
Contributor

@nilason nilason left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me!

@nilason nilason added this to the 8.5.0 milestone Jun 24, 2025
@echoix echoix merged commit 801e552 into OSGeo:main Jun 24, 2025
27 checks passed
@echoix echoix deleted the i18n-string-by-parts branch June 24, 2025 09:57
@marisn
Copy link
Contributor

marisn commented Jun 25, 2025

G_get_units_name is exactly how things should NOT be done. A great bad example.

@echoix
Copy link
Member Author

echoix commented Jun 25, 2025

G_get_units_name is exactly how things should NOT be done. A great bad example.

My eyes were bleeding too

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C Related code is in C database Related to database management display GUI wxGUI related imagery libraries module Python Related code is in Python raster Related to raster data processing translation Message translation related vector Related to vector data processing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants